Skip to content

Conversation

@Turbo87
Copy link
Member

@Turbo87 Turbo87 commented Nov 10, 2024

This unblocks #9879 by adding a couple of basic tests for the route.

Additionally, the route handler is moved to a dedicated module, making it easier to colocate the tests with the implementation in the same file.

@Turbo87 Turbo87 added C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear A-backend ⚙️ labels Nov 10, 2024
@Turbo87 Turbo87 enabled auto-merge November 10, 2024 13:27
@Turbo87 Turbo87 merged commit 9393e1b into rust-lang:main Nov 10, 2024
8 checks passed
@Turbo87 Turbo87 deleted the resend-tests branch November 10, 2024 13:31
@codecov
Copy link

codecov bot commented Nov 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.99%. Comparing base (18b2883) to head (7bf0db5).
Report is 4 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #9888      +/-   ##
==========================================
+ Coverage   88.87%   88.99%   +0.11%     
==========================================
  Files         289      290       +1     
  Lines       30047    30080      +33     
==========================================
+ Hits        26705    26769      +64     
+ Misses       3342     3311      -31     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-backend ⚙️ C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant